Skip to content

improve NaiveIntrospection behavior and execute introspection when needed#802

Merged
bnjjj merged 4 commits intomainfrom
bnjjj/feat_796
Apr 7, 2022
Merged

improve NaiveIntrospection behavior and execute introspection when needed#802
bnjjj merged 4 commits intomainfrom
bnjjj/feat_796

Conversation

@bnjjj
Copy link
Contributor

@bnjjj bnjjj commented Apr 6, 2022

close #796
close #637
close #798

…eded

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@github-actions

This comment has been minimized.

@netlify
Copy link

netlify bot commented Apr 6, 2022

Deploy Preview for apollo-router-docs ready!

Name Link
🔨 Latest commit b135b42
🔍 Latest deploy log https://app.netlify.com/sites/apollo-router-docs/deploys/624ec85a8dcac20008b37bd4
😎 Deploy Preview https://deploy-preview-802--apollo-router-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bnjjj bnjjj requested review from BrynCooke, Geal, garypen and o0Ignition0o and removed request for o0Ignition0o April 6, 2022 16:47
@bnjjj bnjjj self-assigned this Apr 6, 2022
Copy link
Contributor

@garypen garypen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One small comment change would be good.

bnjjj added 3 commits April 7, 2022 13:13
…ction

Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
Signed-off-by: Benjamin Coenen <5719034+bnjjj@users.noreply.github.com>
@bnjjj bnjjj requested a review from Geal April 7, 2022 11:19
@bnjjj bnjjj merged commit 6fc0201 into main Apr 7, 2022
@bnjjj bnjjj deleted the bnjjj/feat_796 branch April 7, 2022 12:21
@Geal Geal added this to the v0.1.0-preview.3 milestone Apr 8, 2022
@Geal Geal mentioned this pull request Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

introspection query failed replacing NaiveIntrospection Introspection query fails with a managed federated supergraph

3 participants